Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

prevent terminating already-terminated process on WSL #861

Closed
wants to merge 1 commit into from

Conversation

ryanfox
Copy link
Collaborator

@ryanfox ryanfox commented Oct 18, 2018

Prevents proc.terminate() from throwing an exception when the process has already terminated.

Potential fix for #765

  • If this is a bugfix, I have provided code that clearly demonstrates the problem and that works when used with this PR
  • I have added a test to the test suite, if necessary
  • I have properly documented new or changed features in the documention, or the docstrings
  • [+ ] I have properly documented unusual changes to the code in the comments around it
  • I have made note of any breaking/backwards incompatible changes

Prevents `proc.terminate()` from throwing an exception when the process has already terminated.

Potential fix for Zulko#765
@OgulcanCelik
Copy link

Just tried on Ubuntu 18.04 Python 3.6.5 on my WSL environment and still getting the ProcessLookupError: [Errno 3] No such process error.

@ryanfox
Copy link
Collaborator Author

ryanfox commented Oct 22, 2018

Can you share the code that is producing the error? What is the full stack trace?

@OgulcanCelik
Copy link

Figured it out. It seems like this fix is just for the video, implemented the same fix for the audio under the pull request: #864

@Overdrivr
Copy link
Collaborator

Hi, thanks for your contribution. Is this a duplicate of #864 ?

@Overdrivr Overdrivr added the duplicate Duplicate of another issue/PR. label Apr 4, 2019
@Overdrivr
Copy link
Collaborator

Fixed by #927

@Overdrivr Overdrivr closed this Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Duplicate of another issue/PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants